Skip to content

internal: No std testing - #6257

Open
Person-93 wants to merge 6 commits into
PyO3:mainfrom
Person-93:no_std_testing
Open

internal: No std testing#6257
Person-93 wants to merge 6 commits into
PyO3:mainfrom
Person-93:no_std_testing

Conversation

@Person-93

@Person-93 Person-93 commented Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This is part of the WIP no_std support.

I am not yet working on CI as that will need a separate discussion so that maintainers can decide how it should be handled. For now, I want to locally run something like this:

PYO3_WIP_NO_STD=1 nox -s test-rust

I've tried making the noxfile aware of the features required for no_std builds, but I am not sure I've got it quite right.

@Person-93 Person-93 changed the title No std testing internal: No std testing Jul 30, 2026
@Person-93
Person-93 marked this pull request as ready for review July 30, 2026 04:30
@Person-93
Person-93 marked this pull request as draft July 30, 2026 05:14
@Person-93
Person-93 marked this pull request as ready for review July 30, 2026 05:27
davidhewitt
davidhewitt approved these changes Aug 5, 2026
View reviewed changes

@davidhewitt davidhewitt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks correct to me!

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 5, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 5, 2026
@Person-93

Person-93 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

I do now know why these changes cause the ffi-check to fail on some platforms. Please help.

@Person-93
Person-93 requested a review from davidhewitt August 5, 2026 20:56
@Person-93 Person-93 mentioned this pull request Aug 5, 2026
Draft
@davidhewitt

davidhewitt commented Aug 6, 2026

Copy link
Copy Markdown
Member

#6302 should have fixed, plz merge main / rebase

@Person-93

Person-93 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

#6302 should have fixed, plz merge main / rebase

@davidhewitt done

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 7, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2026
@Person-93

Person-93 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

@davidhewitt I think that was a spurious issue with cargo-careful.

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 7, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2026
@Person-93

Person-93 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

@davidhewitt, it looks like it's not a spurious issue, but for some reason it's not finding the std that cargo-careful compiled for it.

@davidhewitt

davidhewitt commented Aug 8, 2026

Copy link
Copy Markdown
Member

Will attempt to debug later

@davidhewitt

davidhewitt commented Aug 8, 2026

Copy link
Copy Markdown
Member

Hoping queuing behind #6310 now lands this...

@davidhewitt
davidhewitt added this pull request to the merge queue Aug 8, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 8, 2026
@Person-93

Person-93 commented Aug 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@davidhewitt the latest failure was a network timeout:

Downloading pyrefly (38.3MiB)
  × Failed to download `pyrefly==0.57.1`
  ├─▶ Failed to extract archive:
  │   pyrefly-0.57.1-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
  ├─▶ I/O operation failed during extraction
  ╰─▶ Failed to download distribution due to network timeout. Try increasing
      UV_HTTP_TIMEOUT (current value: 30s).

https://github.com/PyO3/pyo3/actions/runs/31274626505/job/93148274520#step:16:677

@alex
alex added this pull request to the merge queue Aug 8, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 9, 2026
@Person-93

Person-93 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

It looks like an inheritance test failed because references to a type were leaked. Could that have been caused by this PR?
https://github.com/PyO3/pyo3/actions/runs/31284320443/job/93170517972#step:7:8438

@davidhewitt

davidhewitt commented Aug 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Yikes, I'm sorry CI has been so flaky here. I'll try to keep chipping away at whatever is going wrong.

@davidhewitt

davidhewitt commented Aug 9, 2026

Copy link
Copy Markdown
Member

I think #6313 should fix once merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@davidhewitt davidhewitt Awaiting requested review from davidhewitt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Person-93 @davidhewitt